Skip to content

CP-311531: handle both 32-bit and 64-bit unreachable distances in the NUMA matrix#6897

Merged
edwintorok merged 1 commit intoxapi-project:masterfrom
edwintorok:private/edvint/distance
Feb 10, 2026
Merged

CP-311531: handle both 32-bit and 64-bit unreachable distances in the NUMA matrix#6897
edwintorok merged 1 commit intoxapi-project:masterfrom
edwintorok:private/edvint/distance

Conversation

@edwintorok
Copy link
Copy Markdown
Member

@edwintorok edwintorok commented Feb 6, 2026

Also <10 is invalid, which is convenient for detecting the 64-bit unreachable value, but this is already handled.

@edwintorok
Copy link
Copy Markdown
Member Author

Although <10 already had code to handle it in is_numa_matrix_reasonable, so maybe I'll change that to <0.

@edwintorok edwintorok force-pushed the private/edvint/distance branch from 355a33f to 3b0eaca Compare February 6, 2026 16:26
@edwintorok edwintorok requested a review from andyhhp February 6, 2026 16:44
… NUMA matrix

Also <10 is invalid, which is convenient for detecting the 64-bit unreachable
value.
But that is already handled in is_numa_matrix_reasonable, so treat just <0 as
unreachable.

Signed-off-by: Edwin Török <edwin.torok@citrix.com>
@edwintorok edwintorok force-pushed the private/edvint/distance branch from 3b0eaca to 79934fb Compare February 9, 2026 09:38
@edwintorok edwintorok changed the title CP-NNNN: handle both 32-bit and 64-bit unreachable distances in the N… CP-311531: handle both 32-bit and 64-bit unreachable distances in the NUMA matrix Feb 9, 2026
@edwintorok edwintorok marked this pull request as ready for review February 9, 2026 09:38
@edwintorok edwintorok enabled auto-merge February 9, 2026 09:38
Comment thread ocaml/xenopsd/lib/topology.ml
@edwintorok edwintorok added this pull request to the merge queue Feb 10, 2026
Merged via the queue into xapi-project:master with commit 9e3bd27 Feb 10, 2026
16 checks passed
@edwintorok edwintorok deleted the private/edvint/distance branch March 13, 2026 15:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants